[WIP] Add Auth, Security, Audit, and AI plugins for Setup App menu#1230
Merged
xuyushun441-sys merged 3 commits intomainfrom Apr 14, 2026
Merged
[WIP] Add Auth, Security, Audit, and AI plugins for Setup App menu#1230xuyushun441-sys merged 3 commits intomainfrom
xuyushun441-sys merged 3 commits intomainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- Import and register AuthPlugin in createKernel.ts before SetupPlugin - Provide mock auth config for demo environment (secret + baseUrl) - Ensure correct plugin order to avoid 'sys' namespace collision - Remove duplicate setupAppConfig registration from browser.ts, server.ts, and test files - Replace setupAppConfig with AuthPlugin + SetupPlugin in console config - All i18n tests pass successfully Agent-Logs-Url: https://github.com/objectstack-ai/objectui/sessions/ad7046f8-9652-4470-b9fa-c1dbdb78ab9a Co-authored-by: xuyushun441-sys <255036401+xuyushun441-sys@users.noreply.github.com>
Contributor
✅ Console Performance Budget
📦 Bundle Size Report
Size Limits
|
Contributor
|
@claude[agent] 注册登录后,setup还是空的 |
AuthPlugin's init() method tries to contribute menu items to the setupNav service, which is only registered after SetupPlugin's init() runs. The previous order had AuthPlugin loading first, causing it to silently fail when trying to contribute navigation items. Correct order: 1. SetupPlugin (registers setupNav service in init phase) 2. AuthPlugin (contributes to setupNav in init phase) This ensures the Setup App menu is populated with auth-related items: - Users - Organizations - Teams - API Keys - Sessions Agent-Logs-Url: https://github.com/objectstack-ai/objectui/sessions/07edc3ef-a3a9-4926-bd1a-db0c2513b862 Co-authored-by: xuyushun441-sys <255036401+xuyushun441-sys@users.noreply.github.com>
Contributor
Author
Fixed in commit 3b84755. The issue was the plugin loading order - AuthPlugin was trying to contribute menu items before SetupPlugin registered the setupNav service. I've corrected the order so SetupPlugin loads first. The Setup App menu now includes 5 navigation items from AuthPlugin:
All under the "Administration" area. |
Contributor
✅ Console Performance Budget
📦 Bundle Size Report
Size Limits
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.